Skip to content

docs: Update CC Onboarding course steps#71

Merged
sarina merged 2 commits intomainfrom
sarina/202501-cc-onboarding
Dec 1, 2025
Merged

docs: Update CC Onboarding course steps#71
sarina merged 2 commits intomainfrom
sarina/202501-cc-onboarding

Conversation

@sarina
Copy link
Contributor

@sarina sarina commented Dec 1, 2025

No description provided.

@sarina sarina requested a review from kdmccormick December 1, 2025 19:57
</choicegroup>
</multiplechoiceresponse>
<problem display_name="Ensure Your Write Access is Set" markdown="null" showanswer="" submission_wait_seconds="0" weight="1.0"><multiplechoiceresponse>
<p>If you have not already done so, file a <a href="https://github.com/openedx/axim-engineering/issues/new/choose">Axim Access/Config GitHub request</a> for adding access for yourself for whichever repo(s) you should get access to. Include a link to your nomination thread and tag the CC who nominated you.</p><p>Once you've received confirmation that your access is set, please visit each repository you were nominated to be a Core Contributor for and ensure you have write access. This is easy to do by navigating to any open PR and checking that the "merge" button is visible underneath the automated test check reporting.</p><label>I have done this, or it has already been done.</label><choicegroup><choice correct="false"><div>I'll do it later</div></choice><choice correct="true"><div>Yes</div></choice><choice correct="false"><div>I'll never do it</div></choice></choicegroup></multiplechoiceresponse>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as the onboarding runbook is currently written, this creates a catch-22. they can't check their access until they've been granted it, and we don't grant it until the course (including this question) is complete.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ugh! What do you think it should say? maybe just "After your access has been granted, be sure to remember to double-check your access"?

Copy link
Member

@kdmccormick kdmccormick Dec 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that. You could have the answers be:

  • OK, I'll double-check my access as soon as it's granted ✅
  • Nah, I'll just wait until there's a problem ❌

@sarina sarina force-pushed the sarina/202501-cc-onboarding branch from 839d01c to 8487632 Compare December 1, 2025 20:16
Copy link
Member

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! LGTM

<choice correct="false">I'll never do it</choice>
</choicegroup>
</multiplechoiceresponse>
<problem display_name="Ensure Your Write Access is Set" markdown="null" showanswer="" submission_wait_seconds="0" weight="1.0"><multiplechoiceresponse>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the pretty xml formatting was nice, consider preserving that? but OTOH, completely understood if you think it's not worth the effort, you're the one maintaining this--up to you.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At some point, Studio exports quit pretty-printing. I have been just raw-exporting these from Studio. cc @ormsbee

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(I don't do any editing in the XML)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@sarina sarina merged commit 959a6e8 into main Dec 1, 2025
3 checks passed
@sarina sarina deleted the sarina/202501-cc-onboarding branch December 1, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants